Skip to content

Fixed concurrency issues with RegionMigration thread pool#12403

Merged
OneSizeFitsQuorum merged 1 commit intomasterfrom
fix_region_migration_parallel
Apr 24, 2024
Merged

Fixed concurrency issues with RegionMigration thread pool#12403
OneSizeFitsQuorum merged 1 commit intomasterfrom
fix_region_migration_parallel

Conversation

@OneSizeFitsQuorum
Copy link
Copy Markdown
Contributor

The current thread pool is still limited to a single task at most, which makes the parallelism of region migration a big problem

Signed-off-by: OneSizeFitQuorum <tanxinyu@apache.org>
@sonarqubecloud
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 24, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 43.09%. Comparing base (1ce494b) to head (8f5df55).
Report is 3 commits behind head on master.

Files Patch % Lines
.../apache/iotdb/db/service/RegionMigrateService.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #12403      +/-   ##
============================================
- Coverage     43.12%   43.09%   -0.04%     
  Complexity       69       69              
============================================
  Files          3084     3085       +1     
  Lines        190299   190400     +101     
  Branches      22671    22680       +9     
============================================
- Hits          82060    82046      -14     
- Misses       108239   108354     +115     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Collaborator

@liyuheng55555 liyuheng55555 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM !

@OneSizeFitsQuorum OneSizeFitsQuorum merged commit c36f714 into master Apr 24, 2024
@OneSizeFitsQuorum OneSizeFitsQuorum deleted the fix_region_migration_parallel branch April 24, 2024 08:40
HTHou pushed a commit that referenced this pull request Jun 26, 2024
Signed-off-by: OneSizeFitQuorum <tanxinyu@apache.org>
SzyWilliam pushed a commit to SzyWilliam/iotdb that referenced this pull request Nov 26, 2024
Signed-off-by: OneSizeFitQuorum <tanxinyu@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants